Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tower: prepare to release 0.4.10 #608

Merged
merged 2 commits into from
Oct 19, 2021
Merged

tower: prepare to release 0.4.10 #608

merged 2 commits into from
Oct 19, 2021

Conversation

davidpdrsn
Copy link
Member

  • Fix accidental breaking change when using the
    rustdoc::broken_intra_doc_links lint (#605)
  • Clarity that tower's minimum supported rust version is 1.46 (#605)

- Fix accidental breaking change when using the
  `rustdoc::broken_intra_doc_links` lint ([#605])
- Clarity that tower's minimum supported rust version is 1.46 ([#605])

[#605]: #605
@davidpdrsn davidpdrsn added the T-release Topic: releases label Oct 19, 2021
tower/CHANGELOG.md Outdated Show resolved Hide resolved
Copy link
Contributor

@jplatte jplatte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but it would be really nice if #603 could be included in this release 😉

authors = ["Tower Maintainers <[email protected]>"]
license = "MIT"
readme = "README.md"
repository = "https://github.com/tower-rs/tower"
homepage = "https://github.com/tower-rs/tower"
documentation = "https://docs.rs/tower/0.4.9"
documentation = "https://docs.rs/tower/0.4.10"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Like http_root_url, this is also not required 😄
(if you don't see the documentation link on some pages, you're most likely blocking the docs.rs JS script that adds them for crates that don't explicitly set this field)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you wanna do a PR that cleans up this stuff? I believe you made one for Axum as well back in the days. That would be awesome 😊

Copy link
Contributor

@jplatte jplatte Oct 19, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I removed up html_root_url in #603, will add another PR that removes this link (after this PR is merged and I do the necessary rebase).

@davidpdrsn
Copy link
Member Author

@jplatte yeah but I kinda wanna get this out asap

I'm having dinner in town but will merge and publish when I'm back, unless someone beats me to it.

Co-authored-by: Oliver Gould <[email protected]>
@davidpdrsn davidpdrsn merged commit d486564 into master Oct 19, 2021
@davidpdrsn davidpdrsn deleted the tower-0.4.10 branch October 19, 2021 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-release Topic: releases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants